Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Initial quote perf tuning by increasing the distribution percent from 5% to 10% #285

Merged
merged 2 commits into from
Aug 16, 2023
Merged

Initial quote perf tuning by increasing the distribution percent from 5% to 10% #285

merged 2 commits into from
Aug 16, 2023

Conversation

jsy1218
Copy link
Member

@jsy1218 jsy1218 commented Aug 16, 2023 •
edited
Loading

Experiment the quote latency improvement in prod for an hour

mikeki
mikeki reviewed Aug 16, 2023
View reviewed changes
lib/handlers/shared.ts Outdated Show resolved Hide resolved
@jsy1218 jsy1218 self-assigned this Aug 16, 2023
@jsy1218 jsy1218 merged commit cd31cbf into main Aug 16, 2023
3 checks passed
@jsy1218 jsy1218 deleted the jsy1218/distribution-percent-tuning branch August 16, 2023 19:44
jsy1218 added a commit that referenced this pull request Aug 16, 2023
@jsy1218
Revert "Initial quote perf tuning by increasing the distribution perc…
6799619 
…ent from 5% to 10% for the mainnet (#285)"

This reverts commit cd31cbf.
@jsy1218 jsy1218 changed the title Initial quote perf tuning by increasing the distribution percent from 5% to 10% for the mainnet Initial quote perf tuning by increasing the distribution percent from 5% to 10% Aug 16, 2023
jsy1218 added a commit that referenced this pull request Aug 16, 2023
@jsy1218
Revert "Initial quote perf tuning by increasing the distribution perc…
98c8fbe 
…ent from 5% to 10% for the mainnet (#285)" (#286)

This reverts commit cd31cbf.
@jsy1218 jsy1218 mentioned this pull request Aug 22, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikeki mikeki mikeki approved these changes

@andysmith415 andysmith415 Awaiting requested review from andysmith415

@cgkol cgkol Awaiting requested review from cgkol

Assignees

@jsy1218 jsy1218

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jsy1218 @mikeki